Skip to content

fix(release): scope PostgreSQL gosu scan noise - #537

Merged
jeremi merged 1 commit into
mainfrom
codex/scope-postgresql-gosu-scan-beta20
Jul 28, 2026
Merged

jeremi merged 1 commit into
mainfrom
codex/scope-postgresql-gosu-scan-beta20

Conversation

@jeremi

@jeremi jeremi commented Jul 28, 2026

Copy link
Copy Markdown
Member

Summary

  • accept Grype v0.114's empty fix state only as non-fixable metadata; findings with fix versions still block
  • exclude only Go-module version matches whose evidence is exactly /usr/local/bin/gosu in the reviewed, digest-locked PostgreSQL image
  • keep the exemption bound to the current immutable image digest so a repin fails closed
  • keep all PostgreSQL, OS package, other binary, High/Critical, and fixable findings blocking

Evidence

  • exact linux/amd64 gosu 1.19.0 binary from the locked PostgreSQL image: govulncheck v1.6.0 -mode=binary reports 0 reachable vulnerabilities
  • exact pinned Syft/Grype v0.114 report after scoped policy: PostgreSQL advisory policy: blocking=0
  • unreviewed image digest and another Go binary path both remain blocking in focused tests

Verification

  • python3 release/scripts/test_registry_release.py (92 passed)
  • python3 release/scripts/check-gates-inventory.py (168 gates)
  • git diff --check

This narrows a scanner-only external-helper false positive after candidate run 30363651864. Notary, Relay, and all three candidate builds passed in that run.

Signed-off-by: Jeremi Joslin <jeremi@joslin.fr>
@jeremi
jeremi enabled auto-merge (squash) July 28, 2026 14:16
@jeremi
jeremi merged commit 14a13b8 into main Jul 28, 2026
29 checks passed
@jeremi
jeremi deleted the codex/scope-postgresql-gosu-scan-beta20 branch July 28, 2026 14:18

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5cfe9f6677

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

evidence is bound to the reviewed immutable image digest and ordered rootfs
layers instead. The policy contains no accepted-risk mechanism: every fixable
finding and every High or Critical finding blocks the candidate.
layers instead. Every fixable finding and every High or Critical finding blocks

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Add the required DCO sign-off

The reviewed commit has no Signed-off-by: trailer, so it does not satisfy the repository's mandatory DCO policy and will be rejected by the corresponding contribution gate. Recreate the commit with a valid sign-off before merging.

AGENTS.md reference: AGENTS.md:L68-L72

Useful? React with 👍 / 👎.

Comment on lines +211 to +212
if reviewed_gosu_go_metadata(artifact, expected_image):
return False, summary

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Restrict the gosu exemption to reviewed advisories

When a later Grype database reports a newly disclosed, genuinely reachable High/Critical or fixable vulnerability in stdlib or golang.org/x/sys for this same pinned gosu binary, this unconditional return still suppresses it because the exemption checks only the image digest, package name/type, and path—not the vulnerability ID or the reachability evidence. The candidate gate can therefore keep passing the pinned vulnerable image indefinitely; scope the exemption to the exact advisories reviewed as false positives, or regenerate and verify bound reachability evidence for each scan.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant